Skip to content

perf(core): Don't rebuild columns and precompile cell templates each time data is changed #4428

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 2, 2015
Merged

perf(core): Don't rebuild columns and precompile cell templates each time data is changed #4428

merged 1 commit into from
Oct 2, 2015

Conversation

oriondean
Copy link
Contributor

Resolves #4386

This is quite a nice performance improvement in grids that have rapidly updating data

@JLLeitschuh
Copy link
Contributor

Thank you for your pr. I'm going to defer this to the rest of the @angular-ui/grid to review. But I'll add my looks good to me.
If you change your commit message to match these guidelines then it will show up in our changelog.
https://github.com/angular-ui/ui-grid/blob/master/CONTRIBUTING.md#-git-commit-guidelines
If you do choose to change your commit message simply amend your commit locally and then do git push [branch name] --force

Added guard to prevent column set being rebuilt and cell templates being
precompiled everytime grid data changes

Fixes #4386
@oriondean
Copy link
Contributor Author

I've updated the commit message to match the guidelines, let me know if it's still incompatible.

It makes good sense to defer this to the rest of the team. Although I checked myself, I'm still unsure whether anything relies on the column set being rebuilt each time the data changes!

swalters added a commit that referenced this pull request Oct 2, 2015
perf(core): Don't rebuild columns and precompile cell templates each time data is changed
@swalters swalters merged commit 94bbb46 into angular-ui:master Oct 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants